From: Carlos Garnacho <carlosg@gnome.org>
Date: Mon, 24 Nov 2025 16:16:57 +0000 (+0100)
Subject: shellDBus: Add new BrightnessChanged signal to brightness interface
X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~44^2~9
X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=8e1d63029a5a999a65402157cda2ca503eb3866c;p=gnome-shell.git

shellDBus: Add new BrightnessChanged signal to brightness interface

This does actually track user-triggered changes, so other pieces
(i.e. gsd-power) can react to this. This will help in reintroducing
brightness re-normalization when brightness is driven by ALS sensors.

Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/3979>
---

diff --git a/data/dbus-interfaces/org.gnome.Shell.Brightness.xml b/data/dbus-interfaces/org.gnome.Shell.Brightness.xml
index a0e639728..9bfed2505 100644
--- a/data/dbus-interfaces/org.gnome.Shell.Brightness.xml
+++ b/data/dbus-interfaces/org.gnome.Shell.Brightness.xml
@@ -41,5 +41,12 @@
     -->
     <property name="HasBrightnessControl" type="b" access="read"/>
 
+    <!--
+        BrightnessChanged:
+
+        Brightness was changed by the user.
+    -->
+    <signal name="BrightnessChanged" />
+
   </interface>
 </node>
diff --git a/js/ui/shellDBus.js b/js/ui/shellDBus.js
index 17f43e46e..1e5a6a322 100644
--- a/js/ui/shellDBus.js
+++ b/js/ui/shellDBus.js
@@ -602,7 +602,10 @@ export class BrightnessDBus {
         Gio.DBus.session.own_name('org.gnome.Shell.Brightness',
             Gio.BusNameOwnerFlags.NONE, null, null);
 
-        this._manager.connectObject('changed', this._sync.bind(this), this);
+        this._manager.connectObject(
+            'changed', () => this._sync(),
+            'user-update', () => this._userChange(),
+            this);
         this._sync();
     }
 
@@ -616,6 +619,10 @@ export class BrightnessDBus {
             new GLib.Variant('b', this._hasBrightnessControl));
     }
 
+    _userChange() {
+        this._dbusImpl.emit_signal('BrightnessChanged', null);
+    }
+
     SetDimming(enable) {
         this._manager.dimming = enable;
     }